Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrated JDA's command-entities into SlashCommand & SlashCommand.Subcommand & made SmartQueue non-blocking #62

Merged
merged 15 commits into from
Nov 11, 2022

Conversation

jasonlessenich
Copy link
Owner

@jasonlessenich jasonlessenich commented Nov 10, 2022

This PR adds JDA's command-entities into SlashCommand & SlashCommand.Subcommand, which allows the use of, for example, slash commands mentions. In addition to that, this PR also brings some QOL enhancements, like the SlashCommand.SubcommandGroup class which replaces the rather annoying Map<SubcommandGroupData, Subcommand[]> and brings some bug fixes regarding SmartQueue.

Copy link
Collaborator

@DenuxPlays DenuxPlays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hate you and wildcard imports

@DenuxPlays
Copy link
Collaborator

Pls update pr name and/or description because you did a bit more

@DenuxPlays DenuxPlays added enhancement Enhancement to an already existing feature. new feature New feature (request). labels Nov 11, 2022
@jasonlessenich jasonlessenich changed the title Integrated JDA's command-entities into SlashCommand & SlashCommand.Subcommand Integrated JDA's command-entities into SlashCommand & SlashCommand.Subcommand & made SmartQueue non-blocking Nov 11, 2022
@jasonlessenich jasonlessenich added the breaking Contains a backwards incompatible change(s) label Nov 11, 2022
@jasonlessenich jasonlessenich marked this pull request as ready for review November 11, 2022 16:46
Copy link
Collaborator

@DenuxPlays DenuxPlays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great
No wildcard imports 👍

@jasonlessenich jasonlessenich merged commit fe1c8da into v1.6 Nov 11, 2022
@jasonlessenich jasonlessenich deleted the dynxsty/command_reference branch November 11, 2022 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Contains a backwards incompatible change(s) enhancement Enhancement to an already existing feature. new feature New feature (request).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants